Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for unix style paths for secret paths #217

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

tegefaulkes
Copy link
Contributor

@tegefaulkes tegefaulkes commented Jun 26, 2024

Description

This PR addresses allowing spaces and other valid characters within the secret paths.

Issues Fixed

Tasks

  • 1. Update the parser to allow for all valid file paths
  • 2. Add a fast-check test for testing this.

Final checklist

  • Domain specific tests
  • Full tests
  • Updated inline-comment documentation
  • Lint fixed
  • Squash and rebased
  • Sanity check the final build

@tegefaulkes tegefaulkes self-assigned this Jun 26, 2024
Copy link

linear bot commented Jun 26, 2024

@tegefaulkes tegefaulkes mentioned this pull request Jun 26, 2024
8 tasks
@tegefaulkes tegefaulkes force-pushed the feature-eng-335-support-spaces-in-secret-paths branch 2 times, most recently from da6974f to e5cf756 Compare June 28, 2024 07:08
We allow most file paths now including spaces. We exclude `=` to allow the `path=whatever` format and `/` because `isomoric-git` doesn't handle it.

[ci skip]
@tegefaulkes tegefaulkes force-pushed the feature-eng-335-support-spaces-in-secret-paths branch from e5cf756 to b1b5030 Compare June 28, 2024 07:19
@tegefaulkes tegefaulkes merged commit 1545cc5 into staging Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Support spaces in secret paths
1 participant